Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

UIDATIMP-1528 Forward merge of file-splitting/DI Task force work for Poppy. #1471

Merged
merged 36 commits into from
Oct 12, 2023
Merged

UIDATIMP-1528 Forward merge of file-splitting/DI Task force work for Poppy. #1471

merged 36 commits into from
Oct 12, 2023

Conversation

JohnC-80
Copy link
Contributor

@JohnC-80 JohnC-80 commented Sep 29, 2023 •
edited
Loading

Similar to #1462 except this is pointed at the Poppy release...

@JohnC-80
add multipart upload utility file
1e6cd05
@JohnC-80
add composite job status utilities
bbfc863
@JohnC-80
add uploadConfiguration to UploadingJobsContext and Provider
ef68260
@JohnC-80
account for uploadConfiguration/splitStatus in DataFetcher
19c589e
@JohnC-80
update multipart upload to handle multiple simultaneous files
d39304b
@JohnC-80
implement S3-like upload
ac09dd1
@JohnC-80
data fetcher lifecycle adjustment
85f18ec
@JohnC-80
add job parts column to job logs
85c5253
@JohnC-80
run job modal and job profile view with splitting message and job par…
4ecd61f 
…ts column
@JohnC-80
DataFetcher tests
d3fc858
@JohnC-80
translations
2a04d13
@JohnC-80
filename cell formatter
8924158
@JohnC-80
wrap all routes, including settings, in UploadJobsContext
54669f7
@JohnC-80
Composite job details, running jobs
e32ceb7
@JohnC-80
Job Summary and tests
e511ea8
@JohnC-80
util tests
c8fb238
@JohnC-80
tests- uploading job display, file item, uploading job context
0073c97
@JohnC-80
update jobExecution PropTypes
7f46b64
@JohnC-80
job logs container test
4467526
@JohnC-80
view job profile tests
4bfff34
@github-actions
Copy link

github-actions bot commented Sep 29, 2023 •
edited
Loading

Jest Unit Test Statistics

       1 files  ±  0     220 suites  +3   13m 2s ⏱️ - 4m 1s
1 228 tests +84  1 224 ✔️ +83  4 💤 +1  0 ❌ ±0 
1 239 runs  +84  1 235 ✔️ +83  4 💤 +1  0 ❌ ±0 

Results for commit ac415c2. ± Comparison against base commit 8149ece.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Sep 29, 2023 •
edited
Loading

BigTest Unit Test Statistics

0 tests  ±0   0 ✔️ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit ac415c2. ± Comparison against base commit 8149ece.

♻️ This comment has been updated with latest results.

@JohnC-80 JohnC-80 mentioned this pull request Oct 2, 2023
Merged
@JohnC-80 JohnC-80 marked this pull request as ready for review October 4, 2023 18:51
mariia-aloshyna
mariia-aloshyna reviewed Oct 9, 2023
View reviewed changes
package.json Outdated Show resolved Hide resolved
@ncovercash ncovercash self-requested a review October 9, 2023 15:28
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

86.5% 86.5% Coverage
2.4% 2.4% Duplication

@ncovercash ncovercash merged commit c3a2c2f into master Oct 12, 2023
5 checks passed
@ncovercash ncovercash deleted the UIDATIMP-1528 branch October 12, 2023 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mariia-aloshyna mariia-aloshyna mariia-aloshyna approved these changes

@ncovercash ncovercash ncovercash approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@JohnC-80 @ncovercash @mariia-aloshyna